gl renderer: Don't forget the offset when rendering blur nodes

Update Spanish translation

Merge branch 'patch-1' into 'master'

Fix a typo in “popup-context-menu” documentation.

See merge request GNOME/gtk!500

Fix a typo in “popup-context-menu” documentation.

Merge branch 'wip/sadiq/750-gtk4' into 'master'

gtkpopover: Fix popover when parent widget is partially visible

Closes #750

See merge request GNOME/gtk!496

gtkpopover: Fix popover when parent widget is partially visible

fixes https://gitlab.gnome.org/GNOME/gtk/issues/750

Updated Spanish translation

Merge branch 'meson-warning' into 'master'

build: Avoid a Meson deprecation warning

See merge request GNOME/gtk!493

build: Avoid a Meson deprecation warning

Using an empty `configuration_data` object to copy a configuration file
is deprecated since Meson 0.47 (released July 2018); the equivalent
behaviour is available by using `copy: true`.

revealer: Handle child-visible properly

The target position is irrelevant for determining if the child should be
visible. When the current position is 0, it needs to be hidden, period.

Fixes #1355

settings: Don't special case IM modules

The previous fixes made it unnecessary to hardcode IM modules for
different display types. The code now automatically skips system IM
modules for other displays.

immodule: Check if module matches display

When loading IM modules from environment variables or GtkSettings, we
would accept "xim" on Wayland. That is a bad idea.

immodule: Don't load unsupported IM module

The code would technically allow loading the xim module when X11 support
was not compiled in.

This is probably an artificial concern, because it's pretty hard to
compile XIM support without X11 support, but it makes the code clearer,
so there we go.

immodule: Pass a GdkDisplay for default context ID

After all, the context depends on the display - we want to use the
wayland context for Wayland,  xim for X11 and so on.

immulticontext: Don't have a global_context_id

Context IDs are dependant on the display - both because displays can use
different backends, but also because changing the GtkSetting is a
per-display operation.

So just remove the cache.
If it turns out we need a per-display cache, we can add one to
GtkSettings.

a11y: Check display in `*grab_cell_focus()`

Calling the accessibility function `grab_focus()` on a `GtkCell` under
Wayland will cause the client to crash.

This is another case of `gdk_x11_get_server_time()` being called
regardless of the actual windowing backend used.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1507

flowbox: Accept multipress gesture late

So it's able to operate properly with the DnD gesture set by
gtk_drag_source_set(). We usually just react on button release,
that's the right time to claim the gesture.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1557

searchenginequartz: Fix naming of Beagle @ comment

As funny as that is, it's not very instructive. :P

https://gitlab.gnome.org/GNOME/gtk/merge_requests/490#note_400993

[GtkSearchEngineQuartz] Limit the returned hits.

When the result set is larger than the limit.

[GtkSearchEngineQuartz] Resolve the path to a GtkFile* for the hit list.

Instead of a char* path, which the search_hit parser doesn't expect,
causing a crash.

Closes: $https://gitlab.gnome.org/GNOME/gtk/issues/815

Merge branch 'fix-corner-resize' into 'master'

Fix resize corners

See merge request GNOME/gtk!481

Merge branch 'commit-docs' into 'master'

docs: Move commit style docs to the contribution guide

See merge request GNOME/gtk!358

Update Japanese translation

Update Japanese translation

(cherry picked from commit 69ddfcf1ed165eb9b092f4da8a4b7396290487b0)

Updated Lithuanian translation

imcontextwayland: (Hopefully) fix if expression

Update POTFILES.in

buttonbox: Remove redundant if statement

This used to do something in the past.

colorchooser: Remove extra custom color

popover demo: Fix size-allocate callback signature

window: Fix state_flags_changed impl name

This is unrelated to GdkSurface and was changed by accident.

stack: Don't queue an unnecessary redraw

Both queue_allocate and queue_resize already queue a draw.

entry: Remove unused struct

Check for the session bus before using it

Patch by Chris Allen, #475

Merge branch 'wip/nbenitezl/move-bookmark-placeholder-index-4' into 'master'

gtkplacessidebar.c: move bookmark at the placeholder index

See merge request GNOME/gtk!459

Merge branch 'patch-1' into 'master'

Zero-fill new GtkTextIter

See merge request GNOME/gtk!455

Merge branch 'wip/carlosg/imwayland-for-master' into 'master'

Use zwp_text_input_v3 on gtk4

Closes #1317 and #1525

See merge request GNOME/gtk!462

Fix XChangeProperty calls

They must be long...

Closes: #1556

Fix resize corners

I was stuck in an X session and noticed that my resize corners
all got east or north cursors. It turns out that gnome-shell
does not properly advertise support for edge constraints under X11,
and the absence of that makes the code for determining the edge
under the cursor misbehave.

This change should fix that.

Merge branch 'wip/carlosg/revealer-fix-interrupted-animations-master' into 'master'

revealer: Fully set the target state if unmapped during animation

See merge request GNOME/gtk!466

image: Avoid some unnecessary signal connections

Equivalent if fd9879e5ffba2c58b897155b39ff79584b3618e0 but for GtkImage.

image: Indentation fix

gl renderer: Add a clip stack

So we can check that the currently set clip is the first one and now
intersect with it. This first clip is always the entire viewport or the
entire render_area and we don't want to end up drawing things to a
texture because of it.

picture: Avoid some unnecessary signal connections

We don't need to ever invalidate the picture size if the paintable tells
us its size is static. Same for the contents.

image: Remove private struct from public header

CellAccessible: Fix return type of row|column_span

See:
 * c876c74eb73fe6daa279b42f9e0a9566ba1ced87
 * 63e0eb5de5d5a14afadebab31fe679beb6706cb2

Update Catalan translation

docs: Clarify the section on dependencies

docs: Clarify the section on commit access

The section is about rules for direct commit access to the upstream
repository, so let's be more clear about it.

docs: Clarify autoclosing issues syntax

Link to the GitLab documentation, and clarify that if no single commit
in a merge requests closes an issue, you should add a reference to the
issue in the commit message anyway.

docs: Make the contribution guide slightly more friendly

This is an important document for newcomers, so we should err on the
side of being more detailed on what kind of contributions we expect,
and how we expect them.

The text is heavily modelled on the contributing-template by Nadia
Eghbal available here:

  https://github.com/nayafia/contributing-template

docs: Move commit style docs to the contribution guide

There's no point in having a separate file detailing how commits ought
to work, considering we already have a contribution guide.

Update Catalan translation

Update Catalan translation

Update Catalan translation

Merge branch 'fix-typo-win32-compilation-speedup-macro-gtk4' into 'master'

Win32: Fix typo on compilation speedup macro define [Gtk4]

See merge request GNOME/gtk!473

Annotate values of PRIORITY constants

g-ir-scanner incorrectly evaluates macro definition that include
references to other macro definitions. Provide a correct value as an
annotation.

Differences in generated gir files:

```diff
@@ -19017 +19017 @@
-    <constant name="PRIORITY_REDRAW" value="20" c:type="GDK_PRIORITY_REDRAW">
+    <constant name="PRIORITY_REDRAW" value="120" c:type="GDK_PRIORITY_REDRAW">
@@ -74229,3 +74229,3 @@
     </constant>
-    <constant name="PRIORITY_RESIZE" value="10" c:type="GTK_PRIORITY_RESIZE">
+    <constant name="PRIORITY_RESIZE" value="110" c:type="GTK_PRIORITY_RESIZE">
       <doc xml:space="preserve">Use this priority for functionality related to size allocation.
@@ -106786,3 +106786,3 @@
     <constant name="TEXT_VIEW_PRIORITY_VALIDATE"
-              value="5"
+              value="125"
               c:type="GTK_TEXT_VIEW_PRIORITY_VALIDATE">
```

See !472

Update Catalan translation

Win32: Fix typo on compilation speedup macro define

There is a typo, the correct macro to define is WIN32_LEAN_AND_MEAN.
After this change <shellapi.h> must be included in order to use
ExtractIconExW().

mediastream: Fix possible memory leak

@error is (transfer full).  So the error passed should be freed if not used

revealer: Fully set the target state if unmapped during animation

If the revealer is told do animate and then unrealize itself, we do
(correctly) stop the animation, but used to do a shortcut where we
just set the target state as current.

Other things are dependent on the animation properly finishing though,
like the contained widget child visibility. This may lead to inconsistent
state where gtk_revealer_get_child_revealed() returns TRUE but the child
widget is unmapped, or vice-versa.

Fully finish the animation here, so the child state is coherent the next
time the revealer is mapped. We can also skip notifying on the property
since it will be handled by gtk_revealer_set_position().

x11: make the tool lookup dependent on the hw id as well

Tools on the same physical item have the same serial number, so the eraser
and the pen part of a single pen share that serial number. With the current
lookup code, we'll always return whichever tool comes first into proximity.

Change the code to use the hw id in addition to the serial number, this way we
can differ between two tools.

x11: don't add unknown tools to our list

Generic tools (Bamboo, built-in tablets) always have the same serial number
assigned by the wacom driver. This includes the touch tool when the wacom
driver handles the touch evdev node (common where users require the wacom
gestures to work).

When the first device is the touch device, a tool is created with that serial.
All future tools now return the touch tool on lookup since they all share the
same serial number. Worse, this happens *across* devices, so the pen
event node gets assigned the touch tool because they all have the same serial.

Since we don't actually care about the touch as a tool, let's skip any unknown
tool. This captures pads as well.

x11: get the tool type from the wacom driver properties

Any wacom device currently sets the tool type to UNKNOWN. The wacom driver has
a property that exports the tool type as one of stylus, eraser, cursor, pad or
touch. Only three of those are useful here but that's better than having all
of them as unknown.

build: wrap IM protocol building under have_wayland altogether

We poke variables that might not be defined if the wayland backend
is disabled.

imwayland: Respect maximum length of 4000 Bytes on strings being sent.

Hitting the limit will raise protocol errors.

imwayland: Plug leaks

The various strings (pending/current preedit, surrounding, and commit
buffer) are being leaked in the case of GtkIMContext destruction.

imwayland.c: fix formatting

imwayland: rearrange functions to remove prototypes

imwayland: Handle enter and leave events

Before this patch, imwayland would assume that text-input enter and leave events follow the general (wl_keyboard) focus, and was unable to handle the situation where they would not be provided at the same time.

imwayland: Collect return value from ::delete-surrounding signal

There's not much we can do about the signal not being handled, but
we should fetch the return value anyway.

imwayland: Don't reset serial while text-input is alive

The serial number is a persistent property of the text-input object.

imwayland: Fix clearing of preedit text in webkitgtk

Fixes webkitgtk misbehaviour as outlined in https://gitlab.gnome.org/GNOME/gtk/issues/1316#note_312942 , which was introduced in 49b17e6c.
The preedit will be cleared on exit only if it is already present.

imwayland: Fix clearing of preedit text

Fixes terminal emulator misbehaviour as outlined in https://gitlab.gnome.org/GNOME/gtk/issues/1316, which was introduced in 49b17e6c. The original commit cleared preedit text by setting it to an empty string, which still counted as existing preedit. The fix sets preedit string to null, which is correctly understood as not present.

modules: Check current context before retrieving surrounding

There may be situations where this might get called while the
currently focused context just went away (eg. after setting the
text widget unsensitive).

Closes: #1317

imwayland: clear preedit on focus out

imwayland: Add text-input-unstable-v3 support

The wayland input module now represents text-input-unstable-v3 support, while the old module supporting gtk-text-input was renamed to gtkwayland.

gstmedia: Implement error handling.

I wonder who forgot that.
Whoops.

mediastream: Allow multiple calls to gtk_media_stream_error()

Just ignore all further ones.

a11y: Fix function return type

Typo right there.

RGBA: tiny grammar improvements in to_string doc

RGBA: Consistently use “” around inline arg names

instead of being inconsistent and not using them later, which leaves a
bunch of single letters floating among real words, not the prettiest.

RGBA: Fix example to_string output for reality/CSS

* We don't output spaces anywhere in the code, unlike the doc suggested.
* CSS explicitly forbids whitespace between function names and lparens:
  https://stackoverflow.com/questions/13877198

gtkplacessidebar.c: move bookmark at the placeholder index

As that index is set in drag_motion_callback() and visually shown
on the widget as a drop target hint.

https://bugzilla.gnome.org/show_bug.cgi?id=787356
https://gitlab.gnome.org/GNOME/gtk/issues/904

ci: Update Docker image to Fedora 29 and meson to 0.49.0

Zero-fill new GtkTextIter

iter_init_common() is used on uninitialized GtkTextIter, and since neither it
nor its callers initiliaze its padding fields, they contain garbage.

This is a problem for Go - which checks that structs passed to C functions do
not contain pointers to Go-allocated memory - when the garbage happens to be
such a pointer.  Although Go zero-fills all GtkTextIter that it allocates, this
does not help when GTK functions such as insert_pixbuf_or_widget_segment called
for gtk_text_buffer_create_child_anchor copy garbage from their stack-allocated
GtkTextIter into a clean iter.  To work around this a GtkTextIter has to be
discraded after use in text buffer anchor inserting functions:
https://github.com/gotk3/gotk3/pull/307

Replace git.gnome.org by Gitlab URL in <GitRepository> in DOAP file

Replace Bugzilla by Gitlab URL in DOAP file

Remove gtk_widget_show_all() call

Extraneous chunk from 7601bca7587dd29bde3d1554e644a10ae6dafc18.

Cherry picked from gtk-3-24, which has a gtk_widget_show_all() function.

Merge branch 'master' into 'master'

A11y: Add support for AtkTableCell

See merge request GNOME/gtk!411

A11y: Add support for AtkTableCell

placesview: List only available protocols as available

We display a list of supported protocols in the server_addresses_popover.

However, this curated list contains protocols which may or may not be
available, depending on the respective gvfs backend being installed.

So, populate the list only with protocols which are available.

https://gitlab.gnome.org/GNOME/gtk/issues/1476

placesview: Set .error style if unsupported protocol

When the user types an address with a schema that is not supported,
the Connect button doesn't become sensitive, but there is no visible
feedback at all.

This feels unresponsive and leaves the user clueless.

While it doesn't help explain why the address doesn't work, this will
provide a hint that the input was acknowledged but doesn't work.

https://gitlab.gnome.org/GNOME/gtk/issues/1476

Merge branch 'fix-polygon-svg-recolor' into 'master'

icontheme: Recolor <polygon> elements in SVGs too

See merge request GNOME/gtk!443

Merge branch 'demo-combobox-typo-gtk4' into 'master'

demos/gtk-demo/combobox: fix typo

See merge request GNOME/gtk!444

Merge branch 'win32-honest-clipboard-gtk4' into 'master'

GDK W32: Be honest about supported clipboard formats (GTK4)

See merge request GNOME/gtk!399

Revert "Adwaita: GNOME 3.32"

This reverts all GNOME 3.32 styling from master

GDK W32: Always set gtk-font-name to the active UI font. Fixes #1484

This makes apps use "Segoe UI 9" by default instead of whatever matches "Sans 10".
It also cleans up the code and uses some new pango API while at it.

This was previously disabled in 9e686d1fb5cb20 because it led to a poor glyph coverage
on certain versions of Windows which don't default to "Segoe UI 9" (Chinese, Korean, ..)
because the font fallback list was missing in pango.

This is about to get fixed in https://gitlab.gnome.org/GNOME/pango/merge_requests/34
so enable it again when we detect a new enough pango version.

(See !436 for the original MR)

demos/gtk-demo/combobox: fix typo

Fix typo that prevented the P-S submenu from displaying correctly

Updated Slovenian translation